Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
datancoffee
changed the title
bradhe
left a comment
bradhe
left a comment
There was a problem hiding this comment.
I think it looks good! Let's ship it. Also a reminder that we need to document the dbt stuff in docs.tower.dev.
jo-sm
left a comment
jo-sm
left a comment
There was a problem hiding this comment.
I only had a read through the readme.md file, and while I think moving some of the things like how to develop into their own doc are fine, I would highly suggest to remove any Tower marketing sounding stuff from the readme (e.g. "what Tower gives you"). When I look at readmes for tools, while I accept a little bit of non-dev fluff from a tool that's created/maintained by a SaaS company, it's a huge turn off to be sold the product when I just want to know how to use your tool, since I've likely already looked at your website and have a general understanding of the product/tool and just want to use it.
Co-authored-by: Ben Lovell <ben@tower.dev>
datancoffee
deleted the
feature/tow-1575-cli-improve-the-tower-cli-readme
branch
5 participants
Split old README into Overview, Advanced Installation Options, and Contributor Setup.
The Nix and Devenv setup is a little advanced to be on the main README, so, moved to a separate file and linked.
Added Features, Use Cases, Demos, Quickstarts to README